feat: add Dependabot config with grouping and a five-day cooldown - #13
Merged
Conversation
Groups updates per ecosystem so one upstream release does not arrive as three pull requests, with GitHub Actions kept separate because a workflow bump changes what runs in the release path and deserves its own review. Five-day cooldown on every ecosystem. This is a supply-chain control, not a convenience: most malicious npm and PyPI releases in recent memory were detected and yanked within hours, so waiting before adoption neutralises that class of attack without anyone needing to be watching at the right moment. Security updates are advisory-driven and ignore it. Follows webapp's existing convention -- multi-ecosystem-groups, per-update cooldown, `dependencies` label -- rather than inventing a second style. Corrects a mistake in my earlier work. I documented the cooldown as coming from the Renovate presets and warned against enabling Dependabot version updates alongside them. But Renovate is not installed on the organisation: there are zero Renovate pull requests across it, while Dependabot is actively opening them and webapp already ships a dependabot.yml. So the Renovate presets were describing a control that was not running, and the warning had it backwards. The presets stay, since they are valid and express the same policy for any consumer that does adopt Renovate, but README and SECURITY.md now state plainly that Dependabot is what runs here and that the presets require installing the Renovate app. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CEdTd43qLEEE5qCsL1A7gW
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Grouped per ecosystem so one upstream release does not arrive as three pull requests, with GitHub Actions separate — a workflow bump changes what runs in the release path and deserves its own review.
Five-day cooldown on every ecosystem. This is a supply-chain control rather than a convenience: most malicious npm and PyPI releases in recent memory were detected and yanked within hours, so waiting before adoption neutralises that class of attack without anyone needing to be watching at the right moment. Security updates are advisory-driven and ignore it.
Follows webapp's existing convention (
multi-ecosystem-groups, per-updatecooldown,dependencieslabel) rather than inventing a second style./js/#subdirectory; Packagist reads the root/pip install git+…and pre-commit expect root metadata/uses:is SHA-pinned; Dependabot bumps pins with the version comment intactCorrects a mistake in my earlier work
I documented the cooldown as coming from the Renovate presets, and warned against enabling Dependabot version updates alongside them.
Renovate is not installed on the organisation. There are zero Renovate PRs across it, while Dependabot is actively opening them and webapp already ships a
dependabot.yml. So the Renovate presets were describing a control that was not running, and my warning had it backwards.The presets stay — they are valid and express the same policy for any consumer that does adopt Renovate — but
README.mdandSECURITY.mdnow state plainly that Dependabot is what runs here, and that the presets require installing the Renovate app.